Use a St.Bin rather than a St.BoxLayout for calendar popup

Using a horizontal St.BoxLayout for calendar container forces
width-for-height layout on the St.Table child. Since St.Table
is naturally width-for-height, this can trigger bugs and is,
at best, a bit ineffecient. Use a St.Bin instead since we don't
need any BoxLayout features.

https://bugzilla.gnome.org/show_bug.cgi?id=618104

diff --git a/js/ui/panel.js b/js/ui/panel.js
index 34f5696e4ffbcbdb09803b81c9cca5cf99360a7c..632016b25782c3039d5ead861c405e37635d8062 100644 (file)
--- a/js/ui/panel.js
+++ b/js/ui/panel.js
@@ -714,10 +714,10 @@ CalendarPopup.prototype = {
     _init: function() {
         let panelActor = Main.panel.actor;
 
-        this.actor = new St.BoxLayout({ name: 'calendarPopup' });
+        this.actor = new St.Bin({ name: 'calendarPopup' });
 
         this.calendar = new Calendar.Calendar();
-        this.actor.add(this.calendar.actor);
+        this.actor.set_child(this.calendar.actor);
 
         Main.chrome.addActor(this.actor, { visibleInOverview: true,
                                            affectsStruts: false });